Skip to content

Don't run Go tests on documentation changes#18867

Merged
smowton merged 2 commits intomainfrom
smowton/admin/avoid-pointless-test-runs
Feb 26, 2025
Merged

Don't run Go tests on documentation changes#18867
smowton merged 2 commits intomainfrom
smowton/admin/avoid-pointless-test-runs

Conversation

@smowton
Copy link
Copy Markdown
Contributor

@smowton smowton commented Feb 26, 2025

No description provided.

Copilot AI review requested due to automatic review settings February 26, 2025 10:58
@smowton smowton requested a review from a team as a code owner February 26, 2025 10:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Overview

This PR updates the GitHub Actions workflow to exclude running Go tests when only documentation changes occur.

  • Excludes the "go/documentation/**" directory from triggering tests on push events
  • Applies the same exclusion for pull request events

Reviewed Changes

File Description
.github/workflows/go-tests.yml Added exclusion for "go/documentation/**" paths to prevent tests from running when only documentation is changed

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

.github/workflows/go-tests.yml:17

  • [nitpick] The inserted exclusion line contains trailing whitespace. Consider removing the extra spaces for consistency.
      - "!go/documentation/**"      

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@smowton smowton merged commit 9458f07 into main Feb 26, 2025
8 checks passed
@smowton smowton deleted the smowton/admin/avoid-pointless-test-runs branch February 26, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants